Comment that option 'foo=foo' won't work.
authorrobertlipe <robertlipe@f51c46e8-681c-474f-0cfe-069cfd0219fb>
Sun, 11 May 2014 20:30:22 +0000 (20:30 +0000)
committerrobertlipe <robertlipe@f51c46e8-681c-474f-0cfe-069cfd0219fb>
Sun, 11 May 2014 20:30:22 +0000 (20:30 +0000)
gpsbabel/vecs.cc

index e6ed393d8194a49aa1f3c918ad3230cd8e100be5..6a485b2f5d814404b15ff5ea2ea2a350a16aa1e0 100644 (file)
@@ -1149,6 +1149,9 @@ assign_option(const char* module, arglist_t* ap, const char* val)
     return;
   }
 
+  // Fixme - this is probably somewhere between wrong and less than great.  If you have an option "foo" 
+  // and want to set it to the value "foo", this code will prevent that from happening, but we seem to have
+  // code all over the place that relies on this. :-/
   if (case_ignore_strcmp(val, ap->argstring) == 0) {
     c = "";
   } else {